Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EDRD Sprint - 9 Development #1389

Merged
merged 18 commits into from
Nov 11, 2024
Merged

EDRD Sprint - 9 Development #1389

merged 18 commits into from
Nov 11, 2024

Conversation

deepakmulamalla
Copy link
Collaborator

@deepakmulamalla deepakmulamalla commented Nov 7, 2024

EDRD-1031 | Remove EDRD Admin from EDRD PSP Queue
EDRD-1009 | Create additional fields to capture phone and email record for providers
EDRD-931 | As an EDRD Staff I want to be able to create reports to facilitate my work so that I do not require a superuser to create my reports
EDRD – 737 | [Enhancement] One-time dose flag on Medication object
EDRD – 1014 |  
EDRD-1017 | Update tooltip on deceased checkbox
EDRD – 1012 | As an EDRD Staff I want to see Case Type in all of the case list views so I can see if a case is New or Renewal
EDRD -1015 | When using the Create EDRD Case button, cases are being created without Medication Requests and before the workflow is completed
EDRD-1018 |  
EDRD-1010 | Update Account List View
EDRD - 1038 | Update case fits criteria label name
EDRD –  967 | EDRD Case Email Notification
EDRD-1007 | Error messages in the Provider Portal are not correct for Scenario 5 and 6
EDRD-1055(Bug) | When using the Create EDRD Case button, cases are being created without Medication Requests and before the workflow is completed
EDRD-992(Bug) | Risdiplam form fully filled out triggered error after signing off prescriber signature and submitting
EDRD-1052(Bug) | Eculizumab aHUS and PNH Funding Duration for Renewals is incorrect in the Medication record
EDRD-996(Bug) | Assessment Question Responses are not being populated from the submitted form for risdiplam & velaglucerase alfa
EDRD-1055(Bug) | When using the Create EDRD Case button, cases are being created without Medication Requests and before the workflow is completed
EDRD-992(Bug) | Risdiplam form fully filled out triggered error after signing off prescriber signature and submitting
EDRD-1012 | As an EDRD Staff I want to see Case Type in all of the case list views so I can see if a case is New or Renewal
EDRD-996(Bug) | Assessment Question Responses are not being populated from the submitted form for risdiplam & velaglucerase alfa
EDRD-985(Bug) | Special Authority has full access to Account Relationships and is able to View, Edit, and Remove relationships established by EDRD
EDRD-988 | Deceased patient automation is updating the MOH Decision Details in a Cancelled case as Denied

<ValidationRule xmlns="http://soap.sforce.com/2006/04/metadata">
<fullName>EDRD_PhoneNumber_Validation</fullName>
<active>true</active>
<errorConditionFormula>OR(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deepakmulamalla Is this validation rule intended to apply to all accounts (SAT and EDRD)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cgijeffolsen, I’ve updated the validation rule to include the EDRD user role check. Please review, and let me know if there are any further adjustments needed.

Copy link
Collaborator

@cgijeffolsen cgijeffolsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All permission changes are within scope for EDRD. All field changes appropriate to EDRD (none are required). List view changes only affect EDRD list views. Flow and email changes are specific to EDRD.

Clarification needed: EDRD_PhoneNumber_Validation is not specific to EDRD users and will apply to all accounts. Please confirm this is expected.

@deepakmulamalla
Copy link
Collaborator Author

All permission changes are within scope for EDRD. All field changes appropriate to EDRD (none are required). List view changes only affect EDRD list views. Flow and email changes are specific to EDRD.

Clarification needed: EDRD_PhoneNumber_Validation is not specific to EDRD users and will apply to all accounts. Please confirm this is expected.

@cgijeffolsen, I’ve updated the validation rule to include the EDRD user role check. Please review, and let me know if there are any further adjustments needed.

@cgijeffolsen cgijeffolsen self-requested a review November 7, 2024 21:04
Copy link
Collaborator

@cgijeffolsen cgijeffolsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All changes now meet expectation and have no risks to SAT. Approved. @deepakmulamalla @RanadheerRG @NataliaNikishina

@deepakmulamalla
Copy link
Collaborator Author

All changes now meet expectation and have no risks to SAT. Approved. @deepakmulamalla @RanadheerRG @NataliaNikishina

Thanks @cgijeffolsen, for approving the PR.

Copy link
Collaborator

@RanadheerRG RanadheerRG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved @deepakmulamalla

@deepakmulamalla
Copy link
Collaborator Author

@NataliaNikishina, can you approve this PR?

@deepakmulamalla
Copy link
Collaborator Author

Approved @deepakmulamalla

Thanks @RanadheerRG

Copy link
Collaborator

@NataliaNikishina NataliaNikishina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deepakmulamalla approved

Copy link

sonarcloud bot commented Nov 9, 2024

@deepakmulamalla deepakmulamalla merged commit 87e343d into EDRD-Main Nov 11, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants